Skip to content

Conversation

@naya451
Copy link
Contributor

@naya451 naya451 commented Mar 21, 2025

@ghost
Copy link

ghost commented Mar 21, 2025

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-app
Copy link

bedevere-app bot commented Mar 21, 2025

Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool.

If this change has little impact on Python users, wait for a maintainer to apply the skip news label instead.

@eendebakpt
Copy link
Contributor

@naya451 Can you sign the CLA? Clicking on the failing check should direct you to the signing agreement

@naya451
Copy link
Contributor Author

naya451 commented Mar 24, 2025

Is that necessary? Unfortunately I can't add the corresponding email to my account.
But I can commit with another email and create another PR.

@tomasr8
Copy link
Member

tomasr8 commented Mar 24, 2025

Is that necessary? Unfortunately I can't add the corresponding email to my account. But I can commit with another email and create another PR.

You could also change the commit email and force push. This would avoid you having to open another PR

@naya451
Copy link
Contributor Author

naya451 commented Mar 24, 2025

Done, thanks for the tip above!

@gpshead
Copy link
Member

gpshead commented May 19, 2025

thanks!

@gpshead gpshead merged commit c45e661 into python:main May 19, 2025
45 checks passed
@gpshead gpshead added the needs backport to 3.14 bugs and security fixes label May 19, 2025
@miss-islington-app
Copy link

Thanks @naya451 for the PR, and @gpshead for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 19, 2025
…onGH-131536)

Move len boundary assertions before using len.
(cherry picked from commit c45e661)

Co-authored-by: naya451 <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented May 19, 2025

GH-134239 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label May 19, 2025
gpshead pushed a commit that referenced this pull request May 20, 2025
…131536) (GH-134239)

gh-131505: Move len boundary assertions before using len. (GH-131536)

Move len boundary assertions before using len.
(cherry picked from commit c45e661)

Co-authored-by: naya451 <[email protected]>
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants